Skip to content

chore(common): merge e2e tests workflows#1676

Merged
mergify[bot] merged 4 commits intomainfrom
eudelins/chore/821/merge-e2e-tests-workflows
Jan 6, 2026
Merged

chore(common): merge e2e tests workflows#1676
mergify[bot] merged 4 commits intomainfrom
eudelins/chore/821/merge-e2e-tests-workflows

Conversation

@eudelins-zama
Copy link
Copy Markdown
Contributor

@eudelins-zama eudelins-zama commented Dec 29, 2025 •
edited
Loading

Closes https://github.com/zama-ai/fhevm-internal/issues/821

Github now supports more than 10 inputs in reusable workflows (https://github.blog/changelog/2025-12-04-actions-workflow-dispatch-workflows-now-support-25-inputs/), so I updated the e2e tests workflow to add all services' versions as inputs

Run example: https://github.com/zama-ai/fhevm/actions/runs/20578396037/job/59100510651

BLOCKING: this issue is closed in the actionlint repo, but still not deployed in a proper release, so our CI won't pass until then
EDIT: fixed by release of actionlint 1.7.10

@eudelins-zama eudelins-zama self-assigned this Dec 29, 2025
@cla-bot cla-bot Bot added the cla-signed label Dec 29, 2025
@mergify
Copy link
Copy Markdown

mergify Bot commented Dec 29, 2025 •
edited
Loading

🧪 CI Insights

Here's what we observed from your CI run for becb27a.

🟢 All jobs passed!

But CI Insights is watching 👀

Copilot AI reviewed Dec 29, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR consolidates the e2e test workflows by merging test-suite-e2e-tests-mq.yml into test-suite-e2e-tests.yml and updates variable naming conventions across the FHEVM stack. The main changes standardize environment variable names by adding prefixes to distinguish between coprocessor and connector services, and update component versions from v0.10.2 to v0.10.5.

  • Standardized variable naming with COPROCESSOR_ prefix for coprocessor services (e.g., DB_MIGRATION_VERSION → COPROCESSOR_DB_MIGRATION_VERSION)
  • Merged separate workflow files into a single unified e2e test workflow
  • Updated all FHEVM component versions from v0.10.2 to v0.10.5

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
test-suite/fhevm/fhevm-cli Updated environment variable names with COPROCESSOR_ prefix and bumped versions to v0.10.5
test-suite/fhevm/scripts/deploy-fhevm-stack.sh Updated log output to reference new prefixed variable names
test-suite/fhevm/docker-compose/coprocessor-docker-compose.yml Updated Docker image tag references to use new prefixed environment variables
.github/workflows/test-suite-orchestrate-e2e-tests.yml Refactored outputs from arrays to individual named outputs for each component version
.github/workflows/test-suite-e2e-tests.yml Consolidated workflow with expanded inputs and simplified environment variable mapping
.github/workflows/test-suite-e2e-tests-mq.yml Removed redundant workflow file as part of consolidation effort

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/test-suite-e2e-tests.yml Outdated
Comment thread .github/workflows/test-suite-e2e-tests.yml Outdated
Comment thread .github/workflows/test-suite-e2e-tests.yml Outdated
Comment thread .github/workflows/test-suite-e2e-tests.yml Outdated
Comment thread .github/workflows/test-suite-e2e-tests.yml Outdated
@eudelins-zama eudelins-zama force-pushed the eudelins/chore/821/merge-e2e-tests-workflows branch 3 times, most recently from 6285a72 to 601379f Compare December 29, 2025 17:03
Copilot AI reviewed Dec 29, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@eudelins-zama eudelins-zama force-pushed the eudelins/chore/821/merge-e2e-tests-workflows branch from 601379f to 003b824 Compare January 5, 2026 14:59
@eudelins-zama eudelins-zama marked this pull request as ready for review January 5, 2026 15:34
@eudelins-zama eudelins-zama requested a review from a team as a code owner January 5, 2026 15:34
@eudelins-zama eudelins-zama requested a review from Copilot January 5, 2026 15:34
Copilot AI reviewed Jan 5, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/test-suite-orchestrate-e2e-tests.yml Outdated
Comment thread .github/workflows/common-pull-request-lint.yml
@eudelins-zama eudelins-zama force-pushed the eudelins/chore/821/merge-e2e-tests-workflows branch from 1050b8e to 566dad4 Compare January 5, 2026 15:40
leventdem
leventdem previously approved these changes Jan 5, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@leventdem leventdem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@eudelins-zama
Copy link
Copy Markdown
Contributor Author

eudelins-zama commented Jan 5, 2026

@Mergifyio queue

@mergify
Copy link
Copy Markdown

mergify Bot commented Jan 5, 2026 •
edited
Loading

queue

❌ This queue command has been cancelled by a dequeue command

@mergify
Copy link
Copy Markdown

mergify Bot commented Jan 5, 2026 •
edited
Loading

Merge Queue Status

🚫 The pull request has left the queue (rule: main) at 566dad4

This pull request spent 21 minutes 46 seconds in the queue, with no time running CI.
The checks were run on draft #1716.

Required conditions to merge
  • check-success = run-e2e-tests / fhevm-e2e-test
  • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
  • branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
  • any of [🛡 GitHub branch protection]:
    • check-success = common-pull-request/lint (bpr)
    • check-neutral = common-pull-request/lint (bpr)
    • check-skipped = common-pull-request/lint (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = coprocessor-cargo-listener-tests/cargo-tests (bpr)
    • check-neutral = coprocessor-cargo-listener-tests/cargo-tests (bpr)
    • check-success = coprocessor-cargo-listener-tests/cargo-tests (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = coprocessor-cargo-test/cargo-tests (bpr)
    • check-neutral = coprocessor-cargo-test/cargo-tests (bpr)
    • check-success = coprocessor-cargo-test/cargo-tests (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = coprocessor-dependency-analysis/dependencies-check (bpr)
    • check-neutral = coprocessor-dependency-analysis/dependencies-check (bpr)
    • check-success = coprocessor-dependency-analysis/dependencies-check (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = gateway-contracts-deployment-tests/sc-deploy (bpr)
    • check-neutral = gateway-contracts-deployment-tests/sc-deploy (bpr)
    • check-success = gateway-contracts-deployment-tests/sc-deploy (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = kms-connector-tests/test-connector (bpr)
    • check-neutral = kms-connector-tests/test-connector (bpr)
    • check-success = kms-connector-tests/test-connector (bpr)

Reason

Pull request #1676 has been dequeued by a dequeue command

Hint

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.
If you do update this pull request, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

@mergify mergify Bot added the queued label Jan 5, 2026
mergify Bot added a commit that referenced this pull request Jan 5, 2026
@mergify
Merge of #1676
b7c7227
@mergify mergify Bot mentioned this pull request Jan 5, 2026
Closed
59 tasks
@eudelins-zama
Copy link
Copy Markdown
Contributor Author

eudelins-zama commented Jan 5, 2026

@Mergifyio dequeue

@mergify
Copy link
Copy Markdown

mergify Bot commented Jan 5, 2026

dequeue

✅ The pull request has been removed from the queue

@mergify mergify Bot added dequeued and removed queued labels Jan 5, 2026
@eudelins-zama eudelins-zama force-pushed the eudelins/chore/821/merge-e2e-tests-workflows branch from 566dad4 to a670f1c Compare January 5, 2026 17:02
@eudelins-zama eudelins-zama requested a review from Copilot January 5, 2026 17:03
@mergify mergify Bot removed the dequeued label Jan 5, 2026
Copilot AI reviewed Jan 5, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/test-suite-e2e-tests.yml
Comment thread .github/workflows/test-suite-e2e-tests.yml Outdated
@eudelins-zama eudelins-zama force-pushed the eudelins/chore/821/merge-e2e-tests-workflows branch from a670f1c to becb27a Compare January 5, 2026 17:14
@eudelins-zama
Copy link
Copy Markdown
Contributor Author

eudelins-zama commented Jan 6, 2026

@Mergifyio queue

@mergify
Copy link
Copy Markdown

mergify Bot commented Jan 6, 2026 •
edited
Loading

queue

✅ The pull request has been merged automatically

Details

The pull request has been merged automatically at 21fbfbc

@mergify
Copy link
Copy Markdown

mergify Bot commented Jan 6, 2026 •
edited
Loading

Merge Queue Status

✅ The pull request has been merged at becb27a

This pull request spent 19 minutes 20 seconds in the queue, including 16 minutes 15 seconds running CI.
The checks were run on draft #1719.

Required conditions to merge
  • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
  • branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
  • check-success = run-e2e-tests / fhevm-e2e-test
  • any of [🛡 GitHub branch protection]:
    • check-success = common-pull-request/lint (bpr)
    • check-neutral = common-pull-request/lint (bpr)
    • check-skipped = common-pull-request/lint (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = coprocessor-cargo-listener-tests/cargo-tests (bpr)
    • check-neutral = coprocessor-cargo-listener-tests/cargo-tests (bpr)
    • check-success = coprocessor-cargo-listener-tests/cargo-tests (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = coprocessor-cargo-test/cargo-tests (bpr)
    • check-neutral = coprocessor-cargo-test/cargo-tests (bpr)
    • check-success = coprocessor-cargo-test/cargo-tests (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = coprocessor-dependency-analysis/dependencies-check (bpr)
    • check-neutral = coprocessor-dependency-analysis/dependencies-check (bpr)
    • check-success = coprocessor-dependency-analysis/dependencies-check (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = gateway-contracts-deployment-tests/sc-deploy (bpr)
    • check-neutral = gateway-contracts-deployment-tests/sc-deploy (bpr)
    • check-success = gateway-contracts-deployment-tests/sc-deploy (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = kms-connector-tests/test-connector (bpr)
    • check-neutral = kms-connector-tests/test-connector (bpr)
    • check-success = kms-connector-tests/test-connector (bpr)

@mergify mergify Bot added the queued label Jan 6, 2026
mergify Bot added a commit that referenced this pull request Jan 6, 2026
@mergify
Merge of #1676
7abf6cc
@mergify mergify Bot mentioned this pull request Jan 6, 2026
Closed
59 tasks
@mergify mergify Bot merged commit 21fbfbc into main Jan 6, 2026
64 checks passed
@mergify mergify Bot deleted the eudelins/chore/821/merge-e2e-tests-workflows branch January 6, 2026 08:19
@mergify mergify Bot removed the queued label Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@leventdem leventdem leventdem approved these changes

Assignees

@eudelins-zama eudelins-zama

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@eudelins-zama @leventdem